Skip to content

fix: env variable version resolution to match legacy behavior#2139

Closed
imranismail wants to merge 1 commit intoasdf-vm:masterfrom
imranismail:patch-1
Closed

fix: env variable version resolution to match legacy behavior#2139
imranismail wants to merge 1 commit intoasdf-vm:masterfrom
imranismail:patch-1

Conversation

@imranismail
Copy link
Copy Markdown

@imranismail imranismail commented Jun 19, 2025

Summary

We still use shell-lived tool session in our workflows, however when migrating to manually setting the behavior using env var, we noticed this issue.

This fix ensures the pattern matches the legacy shell behavior

upcase_name=$(tr '[:lower:]-' '[:upper:]_' <<<"$plugin")

@imranismail imranismail requested a review from a team as a code owner June 19, 2025 03:34
@imranismail imranismail changed the title Fix variable resolution fix: env variable version resolution to match legacy behavior Jun 19, 2025
@Stratus3D
Copy link
Copy Markdown
Member

Thanks for the PR @imranismail and sorry for the extremely late review! These same changes were merged from #2194 and will go out in the next patch release. Closing this PR as it is no longer needed.

@Stratus3D Stratus3D closed this Apr 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants